Skip to content

[DEV-14329] refactor table spec#4635

Open
james-at-kc wants to merge 33 commits intoqatfrom
ftr/14329-refactor-table-spec
Open

[DEV-14329] refactor table spec#4635
james-at-kc wants to merge 33 commits intoqatfrom
ftr/14329-refactor-table-spec

Conversation

@james-at-kc
Copy link
Copy Markdown

@james-at-kc james-at-kc commented Apr 22, 2026

Description:

Changes table spec from a dictionary to a class to enable type checking.

Technical Details:

  • searched for all instances of table spec
  • examined parameter data types
  • created a class
  • changed dictionary-style references ["fieldname"] to class-style references .fieldname

Requirements for PR Merge:

  1. Unit & integration tests updated
  2. API documentation updated (examples listed below)
    1. API Contracts
    2. API UI
    3. Comments

N/A

  1. Data validation completed (examples listed below)
    1. Does this work well with the current frontend? Or is the frontend aware of a needed change?
    2. Is performance impacted in the changes (e.g., API, pipeline, downloads, etc.)?
    3. Is the expected data returned with the expected format?

N/A

  1. Appropriate Operations ticket(s) created

N/A

  1. Jira Ticket(s)
    1. DEV-14329

Explain N/A in above checklist:

There is no change to the API, nor to the data validation; no impact to the frontend, no change to expected data, and no performance impact. No operations impact.

@james-at-kc james-at-kc marked this pull request as ready for review April 22, 2026 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant